Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[QNN EP] Improve QNN error reporting using the error message #21458

Merged
merged 8 commits into from
Jul 24, 2024
Merged

[QNN EP] Improve QNN error reporting using the error message #21458

merged 8 commits into from
Jul 24, 2024

Conversation

skyline75489
Copy link
Contributor

Description

Massively improve the QNN error reporting by invoking QnnError_getMessage and returning the error message.

Motivation and Context

Example error message before this change:

QNN SetupBackend failed Failed to create device. Error: 14001

After:

QNN SetupBackend failed Failed to create device. Error: QNN_DEVICE_ERROR_INVALID_CONFIG: Invalid config values

@skyline75489 skyline75489 changed the title Improve QNN error reporting by returning the error message [QNN EP] Improve QNN error reporting using the error message Jul 23, 2024
@jywu-msft jywu-msft requested a review from HectorSVC July 23, 2024 17:53
@skyline75489 skyline75489 added the ep:QNN issues related to QNN exeution provider label Jul 24, 2024
@HectorSVC HectorSVC merged commit 6794dfd into main Jul 24, 2024
91 of 97 checks passed
@HectorSVC HectorSVC deleted the jialli/qnn-error-msg branch July 24, 2024 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HectorSVC HectorSVC HectorSVC approved these changes

Assignees
No one assigned
Labels
ep:QNN issues related to QNN exeution provider
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@skyline75489 @HectorSVC